[Feat] 식당 정보 화면 NestedScrollView로 감싸기 #346
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Summary of Changes
Hello @kangyuri1114, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!
이 PR은 식당 정보 화면의 사용자 경험을 향상시키기 위해 스크롤 기능을 추가하고 UI 디자인 가이드라인을 준수하도록 시각적 요소를 개선합니다. 이를 통해 다양한 기기 크기에서 정보가 잘 표시되고, 사용자가 모든 내용을 쉽게 탐색할 수 있도록 합니다.
Highlights
- 스크롤 기능 개선: 식당 정보 화면의 레이아웃을 NestedScrollView로 감싸서 내용이 길어질 경우 스크롤이 가능하도록 개선했습니다.
- UI 일관성 확보: 기존 Figma 디자인에 맞춰 상단에 회색 드래그 핸들(dragHandle)을 추가하여 UI 일관성을 높였습니다.
- 레이아웃 구조 조정: NestedScrollView 도입에 따라 기존 LinearLayout의 속성 및 계층 구조를 조정하여 올바른 렌더링을 보장했습니다.
Using Gemini Code Assist
The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.
Invoking Gemini
You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
Customization
To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.
Limitations & Feedback
Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.
You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.
Footnotes
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
kangyuri1114
changed the title
kangyuri1114
changed the title
There was a problem hiding this comment.
Code Review
안녕하세요. 식당 정보 화면의 UI가 작은 기기에서 잘리는 문제를 NestedScrollView를 사용하여 해결하신 점이 좋습니다. 덕분에 콘텐츠가 길어져도 사용자가 스크롤하여 모든 정보를 확인할 수 있게 되었네요. 또한, 상단에 드래그 핸들을 추가하여 BottomSheet임을 시각적으로 명확하게 알려주는 개선도 훌륭합니다.
코드 리뷰에서는 몇 가지 XML 레이아웃 개선점을 제안했습니다.
ConstraintLayout의 자식이 아닌 뷰에 포함된 불필요한app:layout_*속성들을 제거하여 코드를 더 깔끔하게 만들 수 있습니다.- 레이아웃 미리보기용 텍스트에
android:text대신tools:text를 일관되게 사용하여, 실제 앱 동작과 디자인 타임의 분리를 명확히 하는 것이 좋습니다.
전반적으로 좋은 변경이라고 생각합니다. 제안드린 내용들을 반영하시면 코드가 더욱 견고해질 것입니다.
| android:layout_width="match_parent" | ||
| android:layout_height="wrap_content"> | ||
| android:layout_height="match_parent" | ||
| android:layout_marginBottom="70dp" | ||
| android:gravity="center" | ||
| android:orientation="vertical" | ||
| app:layout_constraintStart_toStartOf="parent" | ||
| app:layout_constraintTop_toTopOf="parent" | ||
| tools:ignore="MissingConstraints"> |
There was a problem hiding this comment.
이 LinearLayout의 부모는 NestedScrollView이므로 ConstraintLayout에서 사용되는 app:layout_constraintStart_toStartOf와 app:layout_constraintTop_toTopOf 속성은 아무런 효과가 없습니다. tools:ignore="MissingConstraints"를 사용하여 경고를 무시하는 대신, 불필요한 속성들을 제거하는 것이 좋습니다.
| android:layout_width="match_parent" | |
| android:layout_height="wrap_content"> | |
| android:layout_height="match_parent" | |
| android:layout_marginBottom="70dp" | |
| android:gravity="center" | |
| android:orientation="vertical" | |
| app:layout_constraintStart_toStartOf="parent" | |
| app:layout_constraintTop_toTopOf="parent" | |
| tools:ignore="MissingConstraints"> | |
| android:layout_width="match_parent" | |
| android:layout_height="match_parent" | |
| android:layout_marginBottom="70dp" | |
| android:gravity="center" | |
| android:orientation="vertical"> |
| android:layout_height="wrap_content" | ||
| android:layout_weight="7" | ||
| android:gravity="end" | ||
| android:text="학생회관 3층" |
There was a problem hiding this comment.
레이아웃 미리보기용 텍스트는 android:text 대신 tools:text를 사용하는 것이 좋습니다. 이렇게 하면 앱 실행 시에는 코드에서 설정한 텍스트가 표시되고, 레이아웃 편집기에서만 미리보기 텍스트가 보이게 됩니다. tv_name 뷰와 일관성을 맞추기 위해 수정하는 것을 권장합니다.
| android:text="학생회관 3층" | |
| tools:text="학생회관 3층" |
| android:id="@+id/iv_cafeteria_photo" | ||
| android:layout_width="match_parent" | ||
| android:layout_height="match_parent" | ||
| android:scaleType="centerCrop" | ||
| android:src="@drawable/img_dodam" | ||
| app:layout_constraintStart_toStartOf="@+id/iv_profile" | ||
| app:layout_constraintTop_toBottomOf="@+id/tv_review_item_comment" /> |
There was a problem hiding this comment.
이 ImageView의 부모는 CardView이며, 이는 FrameLayout을 상속합니다. 따라서 ConstraintLayout에서 사용되는 app:layout_constraintStart_toStartOf와 app:layout_constraintTop_toBottomOf 속성은 여기서 아무런 효과가 없습니다. 불필요한 속성이므로 제거하는 것이 좋습니다.
| android:id="@+id/iv_cafeteria_photo" | |
| android:layout_width="match_parent" | |
| android:layout_height="match_parent" | |
| android:scaleType="centerCrop" | |
| android:src="@drawable/img_dodam" | |
| app:layout_constraintStart_toStartOf="@+id/iv_profile" | |
| app:layout_constraintTop_toBottomOf="@+id/tv_review_item_comment" /> | |
| android:id="@+id/iv_cafeteria_photo" | |
| android:layout_width="match_parent" | |
| android:layout_height="match_parent" | |
| android:scaleType="centerCrop" | |
| android:src="@drawable/img_dodam" /> |
| android:text="08:10~09:20\n | ||
| 천원의 아침밥, 100개 한정\n\n | ||
| 11:20~14:00\n | ||
| 점심\n\n | ||
| 14:00~17:20\n | ||
| 공간 개방" |
There was a problem hiding this comment.
레이아웃 미리보기용 텍스트는 android:text 대신 tools:text를 사용하는 것이 좋습니다. 이렇게 하면 앱 실행 시에는 코드에서 설정한 텍스트가 표시되고, 레이아웃 편집기에서만 미리보기 텍스트가 보이게 됩니다.
tools:text="08:10~09:20
천원의 아침밥, 100개 한정
11:20~14:00
점심
14:00~17:20
공간 개방"
| android:text="3개 코너 운영\n | ||
| 뚝배기, 덮밥, 양식" |
There was a problem hiding this comment.
레이아웃 미리보기용 텍스트는 android:text 대신 tools:text를 사용하는 것이 좋습니다. 이렇게 하면 앱 실행 시에는 코드에서 설정한 텍스트가 표시되고, 레이아웃 편집기에서만 미리보기 텍스트가 보이게 됩니다.
tools:text="3개 코너 운영
뚝배기, 덮밥, 양식"
|
제미나이 리뷰도 반영했습니다! |
Summary
Describe your changes
문제 화면
수정 후
기존 피그마 화면과 동일하지 않은 부분(상단 회색 선 - dragHandle) 도 추가했습니다.
Issue
To reviewers
이부분까지 반영해서 3.0.0 배포할게요